-
Notifications
You must be signed in to change notification settings - Fork 10
chore: Update coderabbitai configuration to not expect docstrings in test #1355
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates the CodeRabbit configuration to exempt Python test files from the requirement to have docstrings, by adding a path-specific review instruction for test paths under package/tests. Flow diagram for CodeRabbit review behavior with test file docstring exemptionflowchart TD
A[PR opened or updated] --> B[CodeRabbit reads .coderabbit.yaml]
B --> C[Identify changed files]
C --> D{File matches path pattern<br/>package/tests/**/*.py?}
D -- Yes --> E[Apply path_instructions<br/>for test files]
E --> F[Do not enforce<br/>docstring requirement<br/>on test functions]
F --> G[Run other enabled<br/>review checks]
D -- No --> H[Apply default<br/>review instructions]
H --> I[Enforce docstring<br/>requirement where applicable]
I --> G
G --> J[Publish GitHub check<br/>with review results]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Warning Ignoring CodeRabbit configuration file changes. For security, only the configuration from the base branch is applied for open source repositories. 📝 WalkthroughWalkthroughA new Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|



#1352 (comment)
Summary by Sourcery
Build:
Summary by CodeRabbit